Compile output files #16

djbe · 2017-01-30T19:54:47Z

Fixes #16.

TODO: find some way to test the custom imports.

…he output file

djbe · 2017-01-30T19:59:01Z

Running this already reveals some issues with the current output:

Colors (rawValue): there's a value that appears twice (0xff66ccff)
Strings (structured/dot): there's a duplicate definition of Deeper (the case, and the sub-enum)
Images (dot-syntax): the struct init methods don't need to be public

AliSoftware

Other than moving the Modules/**/*.swift and Definitions.swift files away from Scripts/, seems ok to me.

AliSoftware · 2017-02-12T21:32:51Z

Scripts/compile-modules.sh

+for m in ${MODULES[@]}; do
+	echo "Compiling module ${m}… (macos)"
+	xcrun $SWIFT3_TOOLCHAIN swiftc -emit-module "Scripts/Modules/$m.swift" -sdk $SDK -module-name "$m" -emit-module-path "Scripts/Modules/swift3" -target $TARGET
+	xcrun $SWIFT2_TOOLCHAIN swiftc -emit-module "Scripts/Modules/$m.swift" -sdk $SDK -module-name "$m" -emit-module-path "Scripts/Modules/swift2.3" -target $TARGET


Not sure if that's logical to put the Modules/*.swift files in Scripts/? Shouldn't they be considered fixtures, or fixture helpers, instead? They are not scripts after all

djbe added 2 commits January 30, 2017 20:52

adjusted compile script to adapt swift version and sdk depending on t…

85d454d

…he output file

travis

34ae68b

djbe added 8 commits January 30, 2017 23:41

change module to have better name

207f58b

Create and compile modules

ac7bc08

Use compiled modules

9c0311f

Define classes for the current module

a570917

compile modules as a rake task

48c4b30

fix module compilation

2169c9c

fix warning about public in internal module

cb20822

Fix duplicate rawcolor value

f67239f

djbe added the type: internal label Jan 31, 2017

djbe mentioned this pull request Jan 31, 2017

Lint output #15

Merged

AliSoftware self-assigned this Feb 12, 2017

AliSoftware reviewed Feb 12, 2017

View reviewed changes

Move env. files to a fixtures subdir

418cd6f

AliSoftware added the status: ready to merge (waiting for CI) label Feb 12, 2017

AliSoftware merged commit 08137d9 into master Feb 12, 2017

djbe deleted the feature/compile-output branch February 12, 2017 22:49

djbe modified the milestone: 1.0.0 Feb 15, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Compile output files #16

Compile output files #16

djbe commented Jan 30, 2017

djbe commented Jan 30, 2017

AliSoftware left a comment

AliSoftware Feb 12, 2017

Compile output files #16

Compile output files #16

Conversation

djbe commented Jan 30, 2017

djbe commented Jan 30, 2017

AliSoftware left a comment

Choose a reason for hiding this comment

AliSoftware Feb 12, 2017

Choose a reason for hiding this comment